Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

support g722-16khz-64kbps audio output format #815

Merged
merged 1 commit into from
Apr 18, 2024
Merged

Conversation

yulin-li
Copy link
Contributor

No description provided.

2,
16,
speechSynthesisOutputFormatString,
speechSynthesisOutputFormatString,
Copy link
Contributor

@BrianMouncer BrianMouncer Apr 18, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

are you sure this second line is correct.
looks like cut and past error, when compared to some of the other formats that have an actual literal format sting in the second position.

e.g. should it have one of these new values.

    [SpeechSynthesisOutputFormat.AmrWb16000Hz]: "amr-wb-16000hz",
    [SpeechSynthesisOutputFormat.G72216Khz64Kbps]: "g722-16khz-64kbps",

Copy link
Contributor Author

@yulin-li yulin-li Apr 18, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for noticing this.

The two formats (customer request to sdk and sdk requests to service) are different if the format has header. E.g., we will replace riff to raw to get the streaming output.

For the new added formats in this PR, they have no header so the two lines should be same

2,
16,
speechSynthesisOutputFormatString,
speechSynthesisOutputFormatString,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same question/comment here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants